Skip to content

docs(guide/dev): remove even more#360

Merged
KSXGitHub merged 1 commit into
remove-some-parts-of-contribution-guidefrom
claude/review-contributing-changes-3PVWc
Mar 18, 2026
Merged

docs(guide/dev): remove even more#360
KSXGitHub merged 1 commit into
remove-some-parts-of-contribution-guidefrom
claude/review-contributing-changes-3PVWc

Conversation

@KSXGitHub

Copy link
Copy Markdown
Owner

Summary

This PR removes several outdated or overly prescriptive style guide sections from the CONTRIBUTING.md file that are no longer aligned with the project's current practices or philosophy.

Changes

  • Removed the section on type aliases using pub use ... as ... for semantic alternative names
  • Removed the "Visibility" section covering pub, pub(crate), and private visibility guidelines
  • Removed the "Struct Field Ordering" section about logical field organization and documentation requirements
  • Removed the "Macros" section about macro usage patterns and scoping

These sections appear to have been superseded by more flexible or implicit conventions, and their removal simplifies the contributing guidelines while maintaining focus on the most important style considerations (derive macro ordering, error handling, etc.).

https://claude.ai/code/session_01VE5W68ky3UobBMongN8Dfc

Remove Visibility, Struct Field Ordering, Macros, and `pub use ... as`
bullet — these are either standard Rust knowledge, generic best practices,
or not used in the codebase.

https://claude.ai/code/session_01VE5W68ky3UobBMongN8Dfc
@github-actions

Copy link
Copy Markdown

Performance Regression Reports

commit: ab830ae

There are no regressions.

@KSXGitHub KSXGitHub marked this pull request as ready for review March 18, 2026 04:09
@KSXGitHub KSXGitHub merged commit e04aedf into remove-some-parts-of-contribution-guide Mar 18, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants